Skip to content

chore: advance sqlfluff sha past click mypy ignore revert (#4967) - #3074

Open
benfdking wants to merge 1 commit into
port/sqlfluff-4975from
port/sqlfluff-4967
Open

chore: advance sqlfluff sha past click mypy ignore revert (#4967)#3074
benfdking wants to merge 1 commit into
port/sqlfluff-4975from
port/sqlfluff-4967

Conversation

@benfdking

Copy link
Copy Markdown
Collaborator

Stacked on sqruff #3072. Merge #3072 first.
Base PR: #3072

Summary

  • Advances .sqlfluff-sha past SQLFluff #4967, which reverts the addition of # type: ignore comments on the click CLI command decorators in src/sqlfluff/__main__.py and src/sqlfluff/cli/commands.py.
  • This is a Python/mypy tooling-only change with no equivalent in sqruff's Rust codebase, so it is a SHA-advance-only port (no code or fixture changes).

Verification

  • cargo build passes (only .sqlfluff-sha changed, so no code impact).
  • Bazel could not run in this environment (network policy blocks the bazel binary download); CI runs the full Bazel suite on this PR.

Ported from SQLFluff 369e7f2d21f50c019211a9c8bca4fbd0cf1a0b2b
sqlfluff/sqlfluff#4967
sqlfluff/sqlfluff@369e7f2


Generated by Claude Code

## Summary
- Advances `.sqlfluff-sha` past SQLFluff #4967, which reverts the addition
  of `# type: ignore` comments on the click CLI command decorators.
- Python/mypy tooling-only change with no equivalent in sqruff's Rust
  codebase, so this is a SHA-advance-only port.

Ported from SQLFluff 369e7f2d21f50c019211a9c8bca4fbd0cf1a0b2b
sqlfluff/sqlfluff#4967
sqlfluff/sqlfluff@369e7f2
@github-actions

github-actions Bot commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

Benchmark for 91fc914

Click to view benchmark
Test Base PR %
DepthMap::from_parent 121.7±0.89µs 120.7±2.49µs -0.82%
fix_complex_query 14.0±0.32ms 14.1±0.15ms +0.71%
fix_superlong 159.8±9.81ms 160.4±9.07ms +0.38%
parse_complex_query 4.2±0.07µs 4.3±0.08µs +2.38%
parse_expression_recursion 7.4±0.23µs 7.4±0.08µs 0.00%
parse_simple_query 1055.5±14.29ns 1052.0±30.89ns -0.33%

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants